From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Thu, 3 Oct 2019 13:13:21 +0000 (+0200)
Subject: lightbox: Show the actor before easing it
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~811^2^2~12
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=92c0171aeb38e4ada0e32c948d074908c8d119f2;p=gnome-shell.git

lightbox: Show the actor before easing it

As per clutter optimizations in should_skip_implicit_transition() any
transition will be ignored if applied to an actor with unmapped clones.

Since we initialize the lightbox as hidden, when we use it standalone (as it
happens for the long fade in screenShield) the transition will be ignored.
This causes the lockscreen fade-out after the idle delay not to work, but
instead to have an apparently locked system that is instead not locked at
all.

So, just ensure that the lightbox actor is visible before applying to it any
transition.

Fixes https://gitlab.gnome.org/GNOME/gnome-shell/issues/1683
---

diff --git a/js/ui/lightbox.js b/js/ui/lightbox.js
index 20092cc04..ecd6304ea 100644
--- a/js/ui/lightbox.js
+++ b/js/ui/lightbox.js
@@ -185,6 +185,8 @@ var Lightbox = class Lightbox {
             this.emit('shown');
         };
 
+        this.actor.show();
+
         if (this._radialEffect) {
             this.actor.ease_property(
                 '@effects.radial.brightness', VIGNETTE_BRIGHTNESS, easeProps);
@@ -197,8 +199,6 @@ var Lightbox = class Lightbox {
                 onComplete
             }));
         }
-
-        this.actor.show();
     }
 
     hide(fadeOutTime) {